Conversation
[FEAT]: 식당(Store) 도메인 엔티티 설계
[FEAT]: HTTPS 적용 및 NGINX Blue-Green 무중단 배포 전환 로직 수정
main 브랜치와 코드 통일
[FIX]: 불필요 도커이미지 제거 명령어 배포 스크립트에 추가
Readme 회원정보 수정
* FEAT: 사업자 인증 시 대표자 성명 입력 필드 추가 및 검증 반영 * FEAT: 가게 등록 시 사업자번호 대표 성명 입력 필드 추가 및 검증 로직 반영
📝 WalkthroughWalkthrough스토어 검색 DTO에 위도(latitude)와 경도(longitude) 필드를 추가하고, 변환기(converter)를 업데이트하여 Store 엔티티에서 이 필드들을 채우도록 변경되었습니다. 사용하지 않는 import도 제거되었습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
sonjunkyu
approved these changes
Feb 19, 2026
SungMinju
approved these changes
Feb 19, 2026
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/main/java/com/eatsfine/eatsfine/domain/store/dto/StoreResDto.java`:
- Around line 31-35: StoreDetailDto is missing latitude/longitude while
StoreSearchDto gained them; add double latitude and double longitude fields to
the StoreDetailDto class (alongside existing fields like mainImageUrl and
isOpenNow), include them in the StoreDetailDto constructor (or builder/factory
such as any existing of()/fromEntity() method), and ensure the mapping code that
builds StoreDetailDto from the Store entity populates these fields; if omission
was intentional, add a clear code comment in StoreDetailDto explaining why
coordinates are omitted.
Comment on lines
+31
to
35
| double latitude, | ||
| double longitude, | ||
| String mainImageUrl, | ||
| boolean isOpenNow | ||
| ){} |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
StoreDetailDto에 좌표 필드 누락 — API 일관성 검토 필요
StoreSearchDto(검색 목록)에는 latitude/longitude가 추가되었지만, StoreDetailDto(상세 조회, line 54-75)에는 없습니다. 클라이언트가 상세 페이지에서 지도 마커를 표시할 때 별도 검색 호출이 필요해지는 문제가 생깁니다. 의도적인 설계라면 코드 주석으로 이유를 명시해두는 것을 권장합니다.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/main/java/com/eatsfine/eatsfine/domain/store/dto/StoreResDto.java` around
lines 31 - 35, StoreDetailDto is missing latitude/longitude while StoreSearchDto
gained them; add double latitude and double longitude fields to the
StoreDetailDto class (alongside existing fields like mainImageUrl and
isOpenNow), include them in the StoreDetailDto constructor (or builder/factory
such as any existing of()/fromEntity() method), and ensure the mapping code that
builds StoreDetailDto from the Store entity populates these fields; if omission
was intentional, add a clear code comment in StoreDetailDto explaining why
coordinates are omitted.
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💡 작업 개요
✅ 작업 내용
🧪 테스트 내용
📝 기타 참고 사항
Summary by CodeRabbit
새로운 기능